MessageTray: remove bad fast path in hideSummaryBoxPointer

The onUngrab callback already checks if all notifications are destroyed and
hides immediately if so. Previous code instead would leave state handling
in an inconsistent state, by not removing the grab, not setting
summaryBoxPointerState to HIDDEN and not disconnecting various signals.

https://bugzilla.gnome.org/show_bug.cgi?id=684036

diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 1b6409871ecb01f76edf871bb8bfffdfa759fa98..21687ee06270da3ee6d83a1c49f44e0e919c01e2 100644 (file)
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -2436,14 +2436,6 @@ const MessageTray = new Lang.Class({
     },
 
     _hideSummaryBoxPointer: function() {
-        // We should be sure to hide the box pointer if all notifications in it are destroyed while
-        // it is hiding, so that we don't show an animation of an empty blob being hidden.
-        if (this._summaryBoxPointerState == State.HIDING &&
-            this._summaryBoxPointerItem.notificationStack.get_n_children() == 0) {
-            this._summaryBoxPointer.actor.hide();
-            return;
-        }
-
         this._grabHelper.ungrab({ actor: this._summaryBoxPointer.bin.child });
     },